Skip to content

Credit new authors and contributors - #100

Open
rcannood wants to merge 10 commits into
mainfrom
chore/credit-missing-contributors
Open

Credit new authors and contributors#100
rcannood wants to merge 10 commits into
mainfrom
chore/credit-missing-contributors

Conversation

@rcannood

@rcannood rcannood commented Aug 13, 2026

Copy link
Copy Markdown
Member

Describe your changes

A handful of people have merged PRs here but weren't credited anywhere yet.

Newly added / changed:

Name Github Contribution Approved
Seo Hyon Kim @seohyonkim DRVI (#61), BRAS (#62), ARI/NMI batch in clustering_overlap (#68) 🆗
Josep Garnica @JGarnica22 ciLISI (#57), STACAS (#58, #85)
Tianyu Liu @HelloWorldLTY CellPLM (#19) 🆗
Calvin McCarter @calvinmccarter ConDo (#83) 🆗
Daniel Schaffer @schafferde Scanorama integrate/correct split (#88)
Stephen Chung @stephen-chung-mh Density-Adaptive BBSG (#94) 🆗
Wendao Liu @liuwd15 FADVI (#90) 🆗
Jérémie Kalfon @jkobject Plenty 🆗

If you created at least 2 PRs you are as an author, otherwise contributor.

@seohyonkim @HelloWorldLTY @JGarnica22 @calvinmccarter @schafferde @stephen-chung-mh @liuwd15 @jkobject @martinkim0 Could you have a quick look at your own row? Happy to fix the spelling, swap in a different name, add an orcid, or take you off entirely if you'd rather not be listed.

I'll leave this open for a week or so before merging. Sorry it took us this long to get you in there!

Checklist before requesting a review

  • I have performed a self-review of my code

  • Check the correct box. Does this PR contain:

    • Breaking changes
    • New functionality
    • Major changes
    • Minor changes
    • Bug fixes
  • Proposed changes are described in the CHANGELOG.md

  • CI Tests succeed and look good!

* use relative __merge__ paths in comp_process_integration.yaml, the
  leading slash made the newer render_readme choke looking for
  /src/api/file_dataset.yaml on disk
* drop --input src/api from create_readme.sh, create_task_readme
  no longer takes it
* bump common submodule
* regenerate readme
* add Seohyon Kim, Tianyu Liu, Josep Garnica, Calvin McCarter, Daniel
  Schaffer, Stephen Chung and Wendao Liu to the authors list
* promote contributors with two or more merged PRs to author
* drop Martin Kim's orcid, it was a copy of Kai Waldrant's
* regenerate readme

@jkobject jkobject left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm!

@stephen-chung-mh

Copy link
Copy Markdown
Contributor

Thanks, my name is correct and my orcid is 0009-0009-5833-9721

Comment thread _viash.yaml
Base automatically changed from fix/render-readme-merge-paths to main August 13, 2026 09:32
Comment thread CHANGELOG.md Outdated
@seohyonkim

Copy link
Copy Markdown
Contributor

Thank you for including me!
name: Seo Hyon Kim
orcid: 0009-0007-3062-4681

@rcannood rcannood changed the title credit missing contributors Credit new authors and contributors Aug 13, 2026
@calvinmccarter

Copy link
Copy Markdown
Contributor

looks good, thanks!

@liuwd15

liuwd15 commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Thanks for including me as a contributor! My ORCID is 0000-0002-5124-9338.

@HelloWorldLTY

Copy link
Copy Markdown
Contributor

Thanks for including me as a contributor! My ORCID is 0000-0002-9412-6573!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants